Bump actions/checkout from 4 to 7 - #1
Conversation
Bumps [actions/checkout](https://github.com/actions/checkout) from 4 to 7. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v4...v7) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: '7' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
Bug #1 (call order): Swap RefreshTabAltShortcutLabels and SyncPaneTabsFromDockChange so labels are assigned AFTER tabs are reordered, not before. This ensures that when a user drag-reorders tabs, the Alt+N shortcut numbers (1-9, 0) correctly reflect the new visual order. Tests added: - DragReorder_WithinSinglePane_LabelsUpdateToReflectNewOrder - DragReorder_ThreeTabs_MoveMiddleToFirst_LabelsCorrect - SplitWorkspace_TwoPanesHorizontal_LeftPaneTabsNumberedFirst - SplitWorkspace_TwoPanesVertical_TopPaneTabsNumberedFirst Bug #2 (multi-pane support): Deferred to follow-up. Current implementation only labels tabs in the first document dock; tabs in secondary split panes within a workspace are not numbered. Full fix requires walking the entire dock tree to enumerate all document docks in visual order. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…ogic Root cause: - Bug #1 (single-pane null labels): RefreshGlobalAltShortcutLabels was checking nonDefaultPanes.Count <= 1 and using foreach loop, which skipped execution when count was 0 (default workspace with no other panes), leaving labels as null. - Bug #2 (simplified global logic): Removed unnecessary FindAllDocumentDocksInVisualOrder and FindAllDocumentDocksRecursive methods. Use existing FindDocumentDock which returns the primary dock for each workspace pane. Changes: - Fixed single-pane fallback logic to handle both 0 and 1 non-default panes correctly - Simplified global tab order to iterate through non-default workspace panes only - Removed unused recursive dock-finding methods Test status: - All single-pane Alt+N label tests now pass (4 tests fixed) - Multi-pane tests still failing due to pre-existing test setup issues (6 remaining) - The 6 failing tests have structural problems (unexpected document counts in docks) that appear unrelated to label assignment logic Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting If you change your mind, just re-open this PR and I'll resolve any conflicts on it. |
|
APPLIED-VIA-FEATURES. This PR was already closed on GitHub, but the bump (actions/checkout v4->v7) has now been applied locally on branch |
… v4->v7 Applies Dependabot #1 and #2 locally via the features integration flow. checkout steps retain submodules: recursive / fetch-depth: 0; each upload-artifact step uses a unique artifact name per run, so the v4+ single-artifact-per-name and no-mutation rules do not affect these workflows (ci.yml, ci-full.yml, publish-validation.yml, release.yml). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Bumps actions/checkout from 4 to 7.
Release notes
Sourced from actions/checkout's releases.
... (truncated)
Changelog
Sourced from actions/checkout's changelog.
... (truncated)
Commits
9c091bbupdate error wording (#2467)1044a6dgetting ready for checkout v7 release (#2464)f028218Bump the minor-npm-dependencies group across 1 directory with 3 updates (#2462)d914b26upgrade module to esm and update dependencies (#2463)537c7efBump@actions/coreand@actions/tool-cacheand Remove uuid (#2459)130a169Bump js-yaml from 4.1.0 to 4.2.0 (#2461)7d09575Bump flatted from 3.3.1 to 3.4.2 (#2460)0f9f3aaBump actions/publish-immutable-action (#2458)f9e715ablock checking out fork pr for pull_request_target and workflow_run (#2454)df4cb1cUpdate changelog for v6.0.3 (#2446)You can trigger a rebase of this PR by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)